@becw I'm trying this on the project I am currently resourced to and will report back. I am going to try with require-dev since we'll only use it on CircleCI and local development.

This is working on my current project. There, I updated the CircleCI config, build config, and build.xml. I added drupal-check to the project's dev dependencies with Composer, and it is working fine there. Once this PR is merged, and we have a new release, I can update it in the project, then remove the explicit dependency from the project, as well as the lines from the build config.

@becw I also ran through the testing steps for a new project and didn't have the same error as @eric-schmidt.

example > drupal-check:

     [echo] $> vendor/bin/drupal-check docroot/modules/custom/
    0 [▓▓▓░░░░░░░░░░░░░░░░░░░░░░░░░]
     [exec] 
     [exec] 
     [exec]  [OK] No errors
     [exec] 

example > drupal-check:

     [echo] $> vendor/bin/drupal-check docroot/themes/custom/
    0 [▓▓▓░░░░░░░░░░░░░░░░░░░░░░░░░]
     [exec] 
     [exec] 
     [exec]  [OK] No errors
     [exec] 

BUILD FINISHED

Total time: 5.4969 seconds

This appears to be an issue with drupal-check itself, as I get the same error when manually running it outside of phing. What is weird is that I can run the exact same command using the exact same version of drupal-check on other sites w/o issue 🤔

@eric-schmidt what version of PHP are you using on your host?

@eric-schmidt @becw I added this to an existing project, and although I had to change my memory_limit in php.ini, it worked as expected. I tried again on a new example project, and it is still working for me there as well.

When using this on an existing project, I encountered the following error:

     [exec]      Child process error (exit code 1): Drupal requires Prophecy PhpUnit when
     [exec]      using PHPUnit 9 or greater. Please use 'composer require --dev
     [exec]      phpspec/prophecy-phpunit:^2' to ensure that it is present.

Once I added the dependency, drupal-check worked as expected.

@becw I added phpspec/prophecy-phpunit:^2 as a dependency of the-build, since that message has come up a few times while using this branch.

The skeleton now installs version 3.0.4 of composer/xdebug-handler, which conflicts with drupal-check dependencies. I ran into this with a project using this branch, so I reran tests here. They failed, as expected. I will look for an update to drupal-check.

composer/composer requires either version 2 or 3 of composer/xdebug-handler, so version 3 gets installed during composer create-project. The composer require step needs to be able to downgrade that to version 2, since drupal-check doesn't support 3 yet. So, I added --with-all-dependencies to the composer require step in the CircleCI build job to allow it to happen.

@becw @eric-schmidt are we ready to merge this yet?